fix: replace async log.warn with sync cliux.print before audit prompt and complete hasFix return - #296
Conversation
cs-raj
commented
Jul 16, 2026
cs-raj
commented
- Replace log.warn (Winston async stream) with cliux.print (process.stdout.write, sync) before cliux.inquire() in module-importer.ts and audit-base-command.ts so the report path message is guaranteed to flush before inquirer takes terminal control
- Add missingMandatoryFields, missingTitleFields, missingMultipleFields to the hasFix return value in audit-base-command.ts — these were checked in the log condition but omitted from the return, silently skipping the import confirmation prompt when only those module types had issues
… and complete hasFix return - Replace log.warn (Winston async stream) with cliux.print (process.stdout.write, sync) before cliux.inquire() in module-importer.ts and audit-base-command.ts so the report path message is guaranteed to flush before inquirer takes terminal control - Add missingMandatoryFields, missingTitleFields, missingMultipleFields to the hasFix return value in audit-base-command.ts — these were checked in the log condition but omitted from the return, silently skipping the import confirmation prompt when only those module types had issues Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
🔒 Security Scan Results
⏱️ SLA Breach Summary
ℹ️ Vulnerabilities Without Available Fixes (Informational Only)The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:
Consider reviewing these vulnerabilities when fixes become available. |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
🔒 Security Scan Results
⏱️ SLA Breach Summary
ℹ️ Vulnerabilities Without Available Fixes (Informational Only)The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:
Consider reviewing these vulnerabilities when fixes become available. |
2 participants
